This pull depends on another pull request for js-multiaddr. [edit] That pull has been merged

Can I get some feedback on this?

@pldespaigne Can you please take a look at this? We'd like to add support to the ENS manager, but we can't until this is merged.

I will look at it ! @ricott1 no offense, but this PR doesn't look clean, do you think a general encoding function will do the job (see #17 ) ?

Absolutely, I'm by no mean an expert developer, I just needed a way to encode onion addresses for the ens-app, but a general encoding would definitely be the way to go.

Hi @pldespaigne , are you working on the general encoding by any chance?

Hi, I don't have so much time currently, but it will be done before Sunday evening !

Hi @ricott1 just merge #28 and publish it to npm as v2.4.1, you should now be able to do contentHash.encode('onion', onionAddress); (onion3 is also supported), I will update the readme later today. Please tell me if it is ok for you so I can close this PR

Thanks!